Skip to content

Pin maturin in CI, add canary workflow, bump crate/recipe version, and fix blocker anchoring logic/tests - #18

Merged
jeromychu23 merged 2 commits into
mainfrom
codex/find-block_key-logic-in-propagate_target_from_ancestor-nmdl1v
Mar 6, 2026
Merged

Pin maturin in CI, add canary workflow, bump crate/recipe version, and fix blocker anchoring logic/tests#18
jeromychu23 merged 2 commits into
mainfrom
codex/find-block_key-logic-in-propagate_target_from_ancestor-nmdl1v

Conversation

@jeromychu23

Copy link
Copy Markdown
Owner

Motivation

  • Pin maturin in CI to ensure reproducible wheel builds across platforms.
  • Add a scheduled canary workflow to exercise the latest allowed maturin to detect upstream regressions early.
  • Bump the crate and packaging version and correct blocker semantics so ancestor blocker checks use the starting row's target as the anchor.

Description

  • Updated multiple GitHub Actions workflows to pass maturin-version: "1.12.5" to PyO3/maturin-action@v1 (.github/workflows/CI.yml, .github/workflows/build-wheels.yml).
  • Added a new scheduled maturin-canary.yml workflow that builds wheels on a small matrix to test the latest allowed maturin.
  • Bumped Rust crate version from 0.1.3 to 0.1.4 in Cargo.toml and updated Cargo.lock, and bumped the conda recipe recipe/meta.yaml version to 0.1.4 and build number to 1.
  • Changed blocker semantics in propagate_target_rows in src/lib.rs to use an anchor_target (the start row's target) for blocker checks across ancestor traversal and removed the initial self-node blocker pre-check, updated the doc comment, and adjusted tests accordingly including adding blocker_checks_use_start_target_as_anchor_across_ancestors and updating the self-blocker test expectation.

Testing

  • Ran cargo test (Rust unit tests) and the test suite completed successfully.

Codex Task

@jeromychu23
jeromychu23 merged commit 8b0b854 into main Mar 6, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant